Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge dev into main: Update Golang version and logging system #438

Merged
merged 2 commits into from
Oct 22, 2024
Merged

Conversation

moonD4rk
Copy link
Owner

@moonD4rk moonD4rk commented Oct 22, 2024

Proposed changes

This PR merges recent changes from the dev branch into the main branch. Key updates include

  • Downgrading Golang version to 1.20 to support Windows 7.
  • Replacing log/slog with the standard logger for improved maintainability and performance

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit, build tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

* chore: downgrade golang version to 1.20, support windows 7

* chore: Update dependencies for Go project.

- Update dependencies in go.sum
- Improvements and optimizations in various files
- Bug fixes and error handling enhancements

* chore: Update modernc.org/sqlite library versions in go.mod and go.sum files

- Update version of `modernc.org/sqlite` to `v1.31.1` in `go.mod` and `go.sum` files
- Update module hash in `go.sum` file for `modernc.org/sqlite`
- Ensure consistency between `go.mod` and `go.sum` files in relation to `modernc.org/sqlite` version

* chore: replace log/slog with standard logger (#436)

* chore: replace log/slog with standard logger

* chore: Update Go dependencies and versions

- Update Go version from `1.22.5` to `1.20` and other dependencies
- Update critical dependencies to latest versions
- Ensure compatibility with new versions of dependencies

* chore: Optimize dependency management in workflows

- Update build and lint workflows to use `go mod tidy` for getting dependencies
- Change modules download mode to `'mod'` in linters configuration
- Add step to get dependencies in lint workflow

* refactor: Update dependencies and refactor Chromium key deletion logic

- Update `modernc.org/sqlite` to `v1.31.1` in `go.mod` and `go.sum`
- Increase version number to `0.5.0` in `cmd/hack-browser-data/main.go`
- Refactor and update logic for filtering and copying items in `browser/chromium/chromium.go`

* Improve logging functionality and data type conversion

- Add `String()` method to `DataType` enum in types.go
- Update log level to Debug in logger_test.go
- Set log level to Debug in `TestLoggerDebug` and `TestLoggerDebugf` functions
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11452927924

Details

  • 91 of 182 (50.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+23.6%) to 42.273%

Changes Missing Coverage Covered Lines Changed/Added Lines %
browser/firefox/firefox.go 0 1 0.0%
browserdata/localstorage/localstorage.go 0 2 0.0%
browserdata/browserdata.go 0 6 0.0%
log/logger.go 83 98 84.69%
log/log.go 0 27 0.0%
types/types.go 8 48 16.67%
Totals Coverage Status
Change from base Build 11105027399: 23.6%
Covered Lines: 372
Relevant Lines: 880

💛 - Coveralls

@moonD4rk moonD4rk merged commit ad020cf into main Oct 22, 2024
8 checks passed
@moonD4rk moonD4rk added this to the v0.5.0 milestone Oct 22, 2024
@moonD4rk moonD4rk deleted the dev branch October 28, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants